Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

EVG-20086 Add task blocked event copy #2021

Merged
merged 2 commits into from
Sep 5, 2023
Merged

Conversation

khelif96
Copy link
Contributor

@khelif96 khelif96 commented Sep 5, 2023

EVG-20086

Description

Added a new event for TASK_BLOCKED. There isn't any additional info in the event that can be used for the copy.

Screenshots

image

@khelif96 khelif96 requested review from Kimchelly and a team September 5, 2023 20:56
Copy link
Contributor

@Kimchelly Kimchelly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM mod trivial wording suggestion.

@@ -17,6 +17,9 @@ export const TaskEventLogLine: React.FC<TaskEventLogEntry> = ({
const containerOrHostCopy = podId ? "container" : "host";
let message: JSX.Element;
switch (eventType) {
case TaskEventType.TaskBlocked:
message = <>This task is blocked.</>;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: the tense of this one is a little different from the other ones already here, should it say something like Task is blocked or Task blocked for consistency with the others?

@khelif96 khelif96 enabled auto-merge September 5, 2023 21:35
@cypress
Copy link

cypress bot commented Sep 5, 2023

Passing run #12474 ↗︎

0 573 7 0 Flakiness 0

Details:

Update copy
Project: Spruce Commit: a41c35be13
Status: Passed Duration: 16:27 💡
Started: Sep 5, 2023 9:23 PM Ended: Sep 5, 2023 9:39 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@khelif96 khelif96 added this pull request to the merge queue Sep 5, 2023
Merged via the queue into evergreen-ci:main with commit e609f98 Sep 5, 2023
2 checks passed
@khelif96 khelif96 deleted the EVG-20086 branch September 5, 2023 21:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants